Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new matric "spatial distortion index" #2260

Merged
merged 41 commits into from
Dec 21, 2023

Conversation

ywchan2005
Copy link
Contributor

@ywchan2005 ywchan2005 commented Dec 4, 2023

What does this PR do?

Add new metric 'D_s' (spatial distortion index)
(being part of #799 )

Before submitting
  • Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?
PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃


📚 Documentation preview 📚: https://torchmetrics--2260.org.readthedocs.build/en/2260/

@ywchan2005 ywchan2005 requested a review from lantiga as a code owner December 4, 2023 21:46
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 4, 2023
@ywchan2005 ywchan2005 changed the title Add new matric "spatial distortion index" WIP: Add new matric "spatial distortion index" Dec 4, 2023
@ywchan2005 ywchan2005 changed the title WIP: Add new matric "spatial distortion index" Add new matric "spatial distortion index" Dec 6, 2023
@ywchan2005 ywchan2005 requested a review from Borda December 6, 2023 11:54
@Borda Borda enabled auto-merge (squash) December 20, 2023 12:31
Copy link

codecov bot commented Dec 20, 2023

Codecov Report

Merging #2260 (5f308ed) into master (b9fe394) will increase coverage by 0%.
The diff coverage is 92%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #2260    +/-   ##
=======================================
  Coverage      87%     87%            
=======================================
  Files         296     298     +2     
  Lines       16709   16854   +145     
=======================================
+ Hits        14501   14634   +133     
- Misses       2208    2220    +12     

@mergify mergify bot added ready and removed has conflicts labels Dec 21, 2023
@Borda Borda merged commit 2cab7b3 into Lightning-AI:master Dec 21, 2023
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation New metric ready topic: Image
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants